Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve ROS2 docs #5628

Merged
merged 2 commits into from
Dec 7, 2023
Merged

Improve ROS2 docs #5628

merged 2 commits into from
Dec 7, 2023

Conversation

Ryanf55
Copy link
Contributor

@Ryanf55 Ryanf55 commented Dec 6, 2023

  • Consistent rosdep instructions
  • Add next steps to make order clearer
  • Put MavProxy in the SITL tutorial
  • Remove specifyinig rosdistro in rosdep call, use the environment variable instead
  • Rely on colcon enabling testing by default
  • Fix invalid link rendering
  • Fix incorrect capitilization of tools

* Consistent rosdep instructions
* Add next steps to make order clearer
* Put MavProxy in the SITL tutorial

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@Hwurzburg
Copy link
Contributor

@Ryanf55 ready to merge?

@Ryanf55
Copy link
Contributor Author

Ryanf55 commented Dec 7, 2023

@Ryanf55 ready to merge?

Yep!

@Hwurzburg Hwurzburg merged commit 907cf20 into ArduPilot:master Dec 7, 2023
4 checks passed
@Ryanf55 Ryanf55 deleted the improve-readme-ros2 branch December 7, 2023 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants